Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add popup menu #67

Closed
wants to merge 7 commits into from
Closed

Add popup menu #67

wants to merge 7 commits into from

Conversation

troyz
Copy link
Contributor

@troyz troyz commented May 4, 2016

I have added the popup menu into the demo application, the document have updated, please reference the section Circle popup menu.

Thanks!

@@ -43,6 +43,13 @@ The implementation of `NADotAnnotation` places a red semi-transparent dot on the

You can find a complete custom annotation example of multi-colored pins in [NAPinAnnotation.h](NAMapKit/NAPinAnnotation.h)/[.m](NAMapKit/NAPinAnnotation.m).

####Circle popup menu
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this should just be added to above, just like we say "you can find a complete custom annotation example in " maybe say "another example of ... can be found in". Make sure to link the source files.

@dblock
Copy link
Collaborator

dblock commented May 4, 2016

Can you please update https://github.com/neilang/NAMapKit/blob/master/CHANGELOG.md, just say that you've added an example that does XYZ. Squash your commits if you don't mind, I'll merge this after, looks great. Thanks for hanging in here!

@troyz
Copy link
Contributor Author

troyz commented May 5, 2016

The README.md and CHANGELOG.md have updated, please correct me if there have any syntax error. :) @dblock

@dblock
Copy link
Collaborator

dblock commented May 5, 2016

Merged via b3b7987, nice job!

@troyz
Copy link
Contributor Author

troyz commented May 5, 2016

Thanks!

@orta
Copy link
Collaborator

orta commented May 6, 2016

👍 closing as merged

@orta orta closed this May 6, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants